Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix user removed from local groups #644

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

adombeck
Copy link
Contributor

@adombeck adombeck commented Nov 19, 2024

Only remove the user from local groups which the user was added to by
authd. Previously, authd removed the user from all local groups which
are not configured in Microsoft Entra, so if the user was added to any
local groups manually, it was removed from those groups again during the
next login.

Closes #576
UDENG-4918

@adombeck adombeck force-pushed the UDENG-4918-do-not-remove-user-from-local-groups branch from 9c134bf to 7217912 Compare November 19, 2024 16:11
Only remove the user from local groups which the user was added to by
authd. Previously, authd removed the user from all local groups which
are not configured in Microsoft Entra, so if the user was added to any
local groups manually, it was removed from those groups again during the
next login.

Closes #576
@adombeck adombeck force-pushed the UDENG-4918-do-not-remove-user-from-local-groups branch from 7217912 to 34bbd33 Compare November 19, 2024 17:52
@adombeck
Copy link
Contributor Author

This is blocked by #641 because I can't update the golden files for the SSH pam integration tests on my local host.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User not able to be added to local groups
1 participant